Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Falling Off Edge in Pet Puzzle #1584

Merged
merged 3 commits into from
May 22, 2024

Conversation

Tiernan-Alderman
Copy link
Contributor

@Tiernan-Alderman Tiernan-Alderman commented May 17, 2024

Issue #547
Description: Modified PetComponents.cpp, dNavMesh.cpp and dNavMesh.h to include a new function nearest point which helps to prevent players from falling off a cliff or an edge when taming a pet. It will instead try to keep the player on the nearest navMesh (terrain)

Motivation and Context: Linked issue from EmosewaMC to work on

Type of Changes: Added new function to dNavMesh and its respective includes in dNavMesh.h. Made changes to petcomponent OnUse (which is used to start a player taming pet interaction).

How Has This Been Tested? : Taming a pet, such as the Goat in Forbidden Valley while on the edge of a cliff, which resulted in keeping the player as close to the edge as necessary to prevent them from falling off. This allows (assuming the camera angle and perspective does not get wacky, which is another WIP). Also testing taming pets right next to ledges, which prevents falling through the wall or clipping through the wall as previously reported.

@Tiernan-Alderman Tiernan-Alderman marked this pull request as ready for review May 17, 2024 05:31
@EmosewaMC EmosewaMC changed the title Falling Off Edge Pet Puzzle Fix fix: Falling Off Edge in Pet Puzzle May 17, 2024
dNavigation/dNavMesh.cpp Outdated Show resolved Hide resolved
@aronwk-aaron aronwk-aaron merged commit d6cac65 into DarkflameUniverse:main May 22, 2024
4 checks passed
@Tiernan-Alderman Tiernan-Alderman deleted the PetPuzzleFix branch May 23, 2024 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants